Skip to content

Conversation

@etr
Copy link
Owner

@etr etr commented Aug 12, 2019

Description of the Change

This change simplifies the checks performed during cross-compilation by removing the compile-and-check performed on libmicrohttpd (which cannot work due to the different binary format)

Possible Drawbacks

This might lead to errors during compilation/linking if the wrong version of libmicrohttpd is used

Verification Process

Unit tests / Integration tests

Release Notes

Removed version checks on libmicrohttpd when cross-compiling.

@etr etr added the build-system Related to the build system. label Aug 12, 2019
@etr etr self-assigned this Aug 12, 2019
@codecov-io
Copy link

Codecov Report

Merging #155 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
+ Coverage    95.3%   95.31%   +<.01%     
==========================================
  Files          35       35              
  Lines        3197     3200       +3     
==========================================
+ Hits         3047     3050       +3     
  Misses        150      150
Impacted Files Coverage Δ
src/httpserver/http_request.hpp 95.83% <0%> (+0.59%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 28288a7...000d07c. Read the comment docs.

@etr etr merged commit 3ffda7b into master Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-system Related to the build system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants